Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Tip Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs). Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
GoodGGori |
kwonkwonn
left a comment
kwonkwonn
left a comment
There was a problem hiding this comment.
UpdateInstance 가 구 스키마 용인지만 확인하고 지우면 될 거 같아요.
| } | ||
|
|
||
| // 현재 미사용중 | ||
| // 이건 미사용중이면 안되지않나? 인스턴스정보 DB에 업데이트 하는거같은데 |
There was a problem hiding this comment.
KWS_Control/structure/control_infra.go
Line 48 in 544f118
<--- 이 친구랑 로직이 어느정도 겹치는 것 같은데 업데이트 전 db 스키마에서 쓰던 것인지, 확인하면 될 것 같아요
| // 현재 미사용 중 | ||
| // 코어 vcpu 갯수 가져오는 함수 | ||
| // 코어에 문의 해봐야함 옛날에 구현 안됬다고 해서 컨트롤에서 9999로 하드코딩 했던거 같음 | ||
| func (c *CoreClient) GetCoreMachineCpuInfo(context context.Context) (*model.CoreMachineCpuInfoResponse, error) { |
There was a problem hiding this comment.
서비스는 구현이 되어있을텐데, 현재 개발환경이 없어서 바로 가져다 쓸 수 있을지는 검증해봐야 됩니당.
그떄 알려드릴게요
2 participants
현재 사용중이지 않은 코드를 삭제하였습니다. 확인 바랍니다